Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add SFR TV services #1248

Merged
merged 15 commits into from
Dec 11, 2023
Merged

Add SFR TV services #1248

merged 15 commits into from
Dec 11, 2023

Conversation

sebmourlhou
Copy link
Contributor

Bonjour,

Je vous propose une PR qui ajoute au plugin les services de SFR TV.
Ceci résout en partie le ticket #1152 dont la cible est plutôt OQEE by Free.

L'ajout du service est plutôt complet :

  • Chaînes en live avec liste des chaînes dynamique en fonction du compte utilisateur
  • Replay
  • Intégration des chaînes à IPTV Manager
  • EPG pour les chaînes principales (celles présentes dans l'XMLTV tv_guide_fr_telerama_{}.xml)
  • Recherche des replay/vidéos

J'ai fait attention à ne rien casser de l'existant (testé avec Nexus et Leia) mais je ne connais peut-être pas toutes les subtilités de CUTVM...
J'ai mis des commentaires là où il y a des particularités.
N'hésitez pas à me faire des retours s'il y a des choses pas claires ou carrément à changer.

En espérant que cela serve à d'autres personnes.

Merci d'avance

@sebmourlhou sebmourlhou mentioned this pull request Nov 24, 2023
Merged
@joaopa00
Copy link
Contributor

joaopa00 commented Nov 25, 2023 •
edited
Loading

Merci pour ce patch vraiemnt très intéresant.

Quelques remarques:
il faudrait plutôt rajouter un menu (dans celui Direct, rattrapges, sites web favoris) un choix fournisseurs internet

Les patches doivent passer l'analyseur de style Flake. Ce n'est pas le cas pour l'instant.
Pour une meilleure lisibilité du ocde, il faut eviter les arguments à rallonges dans l'appel des fonctions.
PAr exemple dans le fichier sfrtv.py ligne 82, il faut mieux quelque chose comme

resp = session.get(ACCESS_TOKEN_URL, params=params, headers=headers max_age=-1)
avec params et headers déclarés avant.

As-tu essayé sur Leia? Si oui arrives-tu à jouer RMCBFMREPLAY sur Leia aussi?
Chez moi sur Leai l'adaptatiion de ton code pour RMCBFMREPLAY ne fonctionne pas sur LEIA.

@sebmourlhou
Copy link
Contributor Author

sebmourlhou commented Nov 25, 2023 •
edited
Loading

J'ai corrigé les warnings des analyseurs de code.

Bonne idée de mettre le service dans un sous-menu "Fournisseurs d'accès internet" !
Je vais voir ça dès que j'ai un peu de temps...

As-tu essayé sur Leia? Si oui arrives-tu à jouer RMCBFMREPLAY sur Leia aussi? Chez moi sur Leai l'adaptatiion de ton code pour RMCBFMREPLAY ne fonctionne pas sur LEIA.

Je viens de retester sur LEIA et RMCBFMREPLAY marche bien chez moi (testé sous windows).
Tu as quelle erreur dans le log ?

@joaopa00
Copy link
Contributor

joaopa00 commented Nov 26, 2023 •
edited
Loading

En effet, j'ai installé Kodi 18 sous windows et cela fonctionne. Le problème vient de Kodi 18 sous Linux. Tout va bien pour ta PR donc

@sebmourlhou
Copy link
Contributor Author

Le service est maintenant dans un sous-menu "Fournisseurs d'accès internet".

En plus des lives, replay et recherche, j'ai ajouté la VOD, ce qui m'a fait faire pas mal de refactoring dans le code.
Note : il faut avoir loué, acheté ou souscrit à un abonnement via le site pour pouvoir voir une vidéo à la demande avec le plugin.

@joaopa00 joaopa00 merged commit 4a77116 into Catch-up-TV-and-More:dev Dec 11, 2023
@sebmourlhou sebmourlhou deleted the sfrtv branch December 14, 2023 15:45
@daufinsyd daufinsyd mentioned this pull request Mar 26, 2024
Closed
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sebmourlhou @joaopa00